Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/generator render widgets #436

Merged
merged 4 commits into from
Nov 20, 2024
Merged

Conversation

lissavxo
Copy link
Collaborator

Related to #426

Description

Created a new field in the generator.html to select whether it will render a paybutton or a paybutton-widget.

This change will help testing, before generator only rendered buttons now it can be used to widgets too.

Test plan

Run yarn dev access http://localhost:10001/paybutton-generator.html test generating widgets/buttons.

@chedieck
Copy link
Collaborator

One thing to notice: this branch has commits from #435, but not all of them (is out-of-date)

When including changes from another branch in the PR branch, try to use the Depends on in the PR description template. This way is easier for reviewers to know which changes to ignore, or which branch to diff it against, or even for you to remember to merge that branch back into this in case that changes (as happened now)

@chedieck chedieck mentioned this pull request Nov 19, 2024
@Klakurka Klakurka merged commit 106411a into master Nov 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants